Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add patches for RA games #385

Merged
merged 1 commit into from
Aug 27, 2024
Merged

Conversation

MrWrightyLC
Copy link
Contributor

@MrWrightyLC MrWrightyLC commented Jun 30, 2024

This PR adds support for patches for some games that are supported on the RetroAchievements site (sometimes some RetroAchievements sets use some games that are modified by xdelta patches, which causes a CRC change)
So using the patches from this same repository of the regular versions to support these versiones with a simple CRC change

I have completed both games to make sure they work.
Persona 3 Fes with a 143+ hour game (using Controllable Characters UNDUB - Original BGM) and I made sure to play the other versions as well for at least 40 minutes.
Raw danger was also played in a game of more than 9 hours (using NTSC Mode + Performance) and I also played the other version for 1 hour.

@MrWrightyLC MrWrightyLC marked this pull request as draft July 9, 2024 00:30
@MrWrightyLC MrWrightyLC marked this pull request as ready for review July 9, 2024 01:04
@F0bes
Copy link
Member

F0bes commented Aug 12, 2024

Could you make the patches use the author field instead of putting the name in the comment?

@MrWrightyLC
Copy link
Contributor Author

MrWrightyLC commented Aug 13, 2024

Could you make the patches use the author field instead of putting the name in the comment?

Done!
Edit: nvm I made a rebase

Add patches for RA games
Raw Danger and Persona 3 FES
@F0bes F0bes merged commit 3fd78ee into PCSX2:main Aug 27, 2024
3 checks passed
@MrWrightyLC MrWrightyLC deleted the again-rcheevos-patches branch August 27, 2024 18:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants